Skip to content

ARTEMIS-5949 Clarify manage permission in default broker.xml#6355

Merged
jbertram merged 1 commit intoapache:mainfrom
anmol-saxena-14:ARTEMIS-5949
Apr 14, 2026
Merged

ARTEMIS-5949 Clarify manage permission in default broker.xml#6355
jbertram merged 1 commit intoapache:mainfrom
anmol-saxena-14:ARTEMIS-5949

Conversation

@anmol-saxena-14
Copy link
Copy Markdown
Contributor

@anmol-saxena-14 anmol-saxena-14 commented Apr 9, 2026

Move the manage permission out of the wildcard match="#" security-setting and into a dedicated match="activemq.management.#" entry, along with the supporting permissions required for management operations (createNonDurableQueue, deleteNonDurableQueue, createAddress, deleteAddress, consume, send).

The old placement was imprecise: granting manage across all addresses is broader than necessary, since it is only meaningful on the management address.
The stale comment explaining the wildcard manage entry is also removed.

Changes:

  • artemis-cli broker.xml template
  • artemis-features artemis.xml
  • jmx-rbac-broker-security smoke-test fixture (aligned to new default)
  • ArtemisTest:add testDefaultSecuritySettings to assert the generated broker.xml has manage only on activemq.management.# and not on #

@jbertram jbertram merged commit 0202f63 into apache:main Apr 14, 2026
6 checks passed
@jbertram
Copy link
Copy Markdown
Contributor

@anmol-saxena-14, thanks for the contribution. Hope to see more from you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants